Skip to content

feat: Add effective map location to metadata events#2321

Merged
rsynek merged 3 commits into
TimefoldAI:mainfrom
diogodanielsoaresferreira:feat/add_effective_map_location_to_insights
Jun 3, 2026
Merged

feat: Add effective map location to metadata events#2321
rsynek merged 3 commits into
TimefoldAI:mainfrom
diogodanielsoaresferreira:feat/add_effective_map_location_to_insights

Conversation

@diogodanielsoaresferreira

Copy link
Copy Markdown
Contributor

Fetch resolved map location from enricher and set it in metadata, so that events have access to the resolved map location.

@rsynek rsynek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Good to merge after exploring if we can introduce some reasonable test coverage the SonarCloud complains about.

To be fair, there are tests in the other related PR.

@sonarqubecloud

sonarqubecloud Bot commented Jun 3, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
44.3% Coverage on New Code (required ≥ 70%)

See analysis details on SonarQube Cloud

@rsynek rsynek merged commit 71f89e1 into TimefoldAI:main Jun 3, 2026
18 of 19 checks passed
rsynek added a commit to rsynek/timefold-solver that referenced this pull request Jun 17, 2026
rsynek added a commit that referenced this pull request Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants